Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes for building with Ant #22

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

yuuyins
Copy link

@yuuyins yuuyins commented Feb 25, 2022

I was getting many errors to build with Ant (see https://bin.disroot.org/?666e30603a62cf2d#HSCCA7wKR1ohQXJahwvy6XeqaHJyAJsovHnQxP69cm77).

So I had to make some small changes for being able to build. The most worth explaining is about the params in the documentation comments, many were not declared which was causing errors in the build, so I removed them.

@guesant
Copy link

guesant commented Feb 25, 2022

@yuuyins I'm (trying to package) packaging as flatpak (#21) and even with the errors you pointed I still can build successfully.

Peek.24-02-2022.21-56.mp4

Build command:

ant jar

@yuuyins
Copy link
Author

yuuyins commented Apr 11, 2022

@guesant pode ser alguma configuração do Ant no Nix, não sei dizer... mas o patch acaba que sendo também uma melhoria pequena geral, então seria legal que o @chcandido desse uma olhada.

aliás, agora o brmodeo já tá no repositório do Nix pra instalar em distribuições Linux que rodam o Nix e no MacOS. NixOS/nixpkgs@7203cae

@guesant
Copy link

guesant commented Apr 11, 2022

@yuuyins q da hora, eu n conhecia esse nix...

@yuuyins
Copy link
Author

yuuyins commented Apr 11, 2022

@guesant é bem util pra desenvolvimento também, uma alternativa ao docker... no momento to tentando empacotar varios apps de engenharia de software, ciencia da computacao, SI la NixOS/nixpkgs#164019

brmodelo foi o primeiro ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants